Add procedure for converting external Swift storage nodes by cschwede · Pull Request #1342 · openstack-k8s-operators/data-plane-adoption

cschwede · 2026-04-01T08:16:37Z

Adds documentation for converting existing TripleO-managed Swift storage nodes post-adoption without data migration.

There are two ways to adopt existing Swift storage nodes:

Migrate data to PVs (or dataplane nodes), which is required if Swift storage was deployed on previous controller nodes
Convert existing TripleO/Director-managed nodes to dataplane nodes, which should be used for any former external ObjectStorage node to avoid moving huge amounts of data.

This commit adds documentation for the second option; the first one already exists.

softwarefactory-project-zuul · 2026-04-02T16:07:29Z

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/data-plane-adoption for 1342,8716ee680e590e4a606c865cb6d677c55767d612

softwarefactory-project-zuul · 2026-04-02T16:10:16Z

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/data-plane-adoption for 1342,0bdade46976b1d2225981766ae1b80aa8974c191

klgill

Hi @cschwede. I have a few style changes that need to be implemented.

cschwede · 2026-04-04T08:33:33Z

Hi @cschwede. I have a few style changes that need to be implemented.

Thanks @klgill for your review! I applied your changes and squashed them into the first commit. I added a second commit with a few additions/fixes, can you have a look at that one too please? I'll squash the commit as well once you reviewed these changes.

klgill

@cschwede I have a few more minor changes.

cschwede · 2026-04-07T08:23:25Z

@cschwede I have a few more minor changes.

Thanks @klgill, applied all suggestions and squashed them into the main commit.

Changes from yesterdays commit: https://github.com/openstack-k8s-operators/data-plane-adoption/compare/b7818a8d98e0dfe8232b6b869f2201ab455c4117..df52575eed95d484bfcf6bf22e861e6085303a82

klgill

I have one minor formatting change. One of the notes is not rendering properly in the preview.

cschwede · 2026-04-07T14:35:42Z

I have one minor formatting change. One of the notes is not rendering properly in the preview.

Thanks, I just committed & squashed the change.

klgill · 2026-04-07T14:37:47Z

@cschwede My latest suggestion didn't work either. I'll need to look at it on my own after this is merged.
Is this PR intended for FR6? If so, can you provide the link to the feature associated with this PR?

cschwede · 2026-04-07T14:47:04Z

@cschwede My latest suggestion didn't work either. I'll need to look at it on my own after this is merged. Is this PR intended for FR6? If so, can you provide the link to the feature associated with this PR?

@klgill Yes, FR6 - this is the feature: OSPRH-23099

Adds documentation for converting existing TripleO-managed Swift storage nodes post-adoption without data migration. There are two ways to adopt existing Swift storage nodes: 1. Migrate data to PVs (or dataplane nodes), which is required if Swift storage was deployed on previous controller nodes 2. Convert existing TripleO/Director-managed nodes to dataplane nodes, which should be used for any former external ObjectStorage node to avoid moving huge amounts of data. This commit adds documentation for the second option; the first one already exists. JIRA: OSPRH-23099 Signed-off-by: Christian Schwede <cschwede@redhat.com> Co-authored-by: Katie Gilligan <kgilliga@redhat.com>

cschwede · 2026-04-07T15:15:56Z

@cschwede My latest suggestion didn't work either. I'll need to look at it on my own after this is merged.

@klgill I think the issue was the intendation: https://github.com/openstack-k8s-operators/data-plane-adoption/compare/dedf9bbcc91a531915e172da698cd74a6cb12886..9fdeaabd9ca139bddd841985e7ee29534551ecfa

klgill · 2026-04-07T15:22:58Z

@cschwede My latest suggestion didn't work either. I'll need to look at it on my own after this is merged.

@klgill I think the issue was the intendation: https://github.com/openstack-k8s-operators/data-plane-adoption/compare/dedf9bbcc91a531915e172da698cd74a6cb12886..9fdeaabd9ca139bddd841985e7ee29534551ecfa

Yes, that looks better!

klgill

/lgtm

openshift-ci · 2026-04-07T15:24:10Z

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

cschwede force-pushed the swift-conversion branch from 8e5a2d1 to 0bdade4 Compare April 1, 2026 08:29

openshift-ci Bot added the needs-rebase label Apr 2, 2026

cschwede force-pushed the swift-conversion branch from 8716ee6 to 0bdade4 Compare April 2, 2026 16:10

cschwede force-pushed the swift-conversion branch from 0bdade4 to 45137bb Compare April 2, 2026 16:18

openshift-ci Bot removed the needs-rebase label Apr 2, 2026

klgill requested changes Apr 3, 2026

View reviewed changes

cschwede force-pushed the swift-conversion branch from 84c7b1b to 2437811 Compare April 4, 2026 08:15

cschwede force-pushed the swift-conversion branch from 2437811 to b7818a8 Compare April 6, 2026 08:32

klgill requested changes Apr 6, 2026

View reviewed changes

cschwede force-pushed the swift-conversion branch 2 times, most recently from 59c7186 to df52575 Compare April 7, 2026 08:20

klgill reviewed Apr 7, 2026

View reviewed changes

Comment thread docs_user/modules/proc_converting-object-storage-nodes.adoc Outdated

klgill requested changes Apr 7, 2026

View reviewed changes

cschwede force-pushed the swift-conversion branch from 98f987d to dedf9bb Compare April 7, 2026 14:34

cschwede force-pushed the swift-conversion branch from dedf9bb to 9fdeaab Compare April 7, 2026 15:13

klgill approved these changes Apr 7, 2026

View reviewed changes

openshift-ci Bot assigned klgill Apr 7, 2026

openshift-ci Bot added the lgtm label Apr 7, 2026

klgill added the approved label Apr 7, 2026

openshift-merge-bot Bot merged commit bf6e42f into openstack-k8s-operators:main Apr 7, 2026
6 checks passed

cschwede deleted the swift-conversion branch April 7, 2026 15:28

Uh oh!

Conversation

cschwede commented Apr 1, 2026

Uh oh!

softwarefactory-project-zuul Bot commented Apr 2, 2026

Uh oh!

softwarefactory-project-zuul Bot commented Apr 2, 2026

Uh oh!

klgill left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cschwede commented Apr 4, 2026

Uh oh!

klgill left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cschwede commented Apr 7, 2026

Uh oh!

Uh oh!

klgill left a comment

Choose a reason for hiding this comment

Uh oh!

cschwede commented Apr 7, 2026

Uh oh!

klgill commented Apr 7, 2026

Uh oh!

cschwede commented Apr 7, 2026

Uh oh!

cschwede commented Apr 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

klgill commented Apr 7, 2026

Uh oh!

klgill left a comment

Choose a reason for hiding this comment

Uh oh!

openshift-ci Bot commented Apr 7, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

cschwede commented Apr 7, 2026 •

edited

Loading